Skip to content

Conversation

@Enselic
Copy link
Member

@Enselic Enselic commented Oct 24, 2025

To avoid linker errors like

relocation truncated to fit: R_AARCH64_ADR_PREL_PG_HI21 against `.data.rel.ro..L.hwasan'

we need to have -C target-feature=+tagged-globals, which is documented here. I learned that here.

Closes #83989

try-job: aarch64-gnu

@rustbot rustbot added PG-exploit-mitigations Project group: Exploit mitigations S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Oct 24, 2025
@Enselic
Copy link
Member Author

Enselic commented Oct 24, 2025

@bors try

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Oct 24, 2025
tests/ui/sanitizer/hwaddress.rs: Run on aarch64 and remove cgu hack

try-job: aarch64-gnu
@rust-log-analyzer

This comment has been minimized.

@rust-bors

This comment was marked as outdated.

@Enselic

This comment was marked as outdated.

rust-bors bot added a commit that referenced this pull request Oct 24, 2025
tests/ui/sanitizer/hwaddress.rs: Run on aarch64 and remove cgu hack

try-job: aarch64-gnu
@rust-bors

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-bors

This comment was marked as outdated.

@Enselic

This comment was marked as outdated.

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Oct 24, 2025
tests/ui/sanitizer/hwaddress.rs: Run on aarch64 and remove cgu hack

try-job: aarch64-gnu
@rust-log-analyzer

This comment has been minimized.

@rust-bors

This comment was marked as outdated.

To avoid linker errors like

    relocation truncated to fit: R_AARCH64_ADR_PREL_PG_HI21 against `.data.rel.ro..L.hwasan'

we need to have `-C target-feature=+tagged-globals`, which is documented here:
https://doc.rust-lang.org/beta/unstable-book/compiler-flags/sanitizer.html#hwaddresssanitizer
@Enselic

This comment was marked as outdated.

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Oct 24, 2025
tests/ui/sanitizer/hwaddress.rs: Run on aarch64 and remove cgu hack

try-job: aarch64-gnu
@rust-bors
Copy link

rust-bors bot commented Oct 24, 2025

☀️ Try build successful (CI)
Build commit: 17d9639 (17d9639cb3c83c378f550f5436c615028128329d, parent: ab925646fae038b02bd462cd328ae9eef1639236)

@Enselic
Copy link
Member Author

Enselic commented Oct 24, 2025

Try build succeeded, so ready for review.

r? @nagisa since you reviewed the original PR and hopefully is comfortable to also review this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PG-exploit-mitigations Project group: Exploit mitigations S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI test sanitize/hwaddress.rs fails with linker error on aarch64 when executed with 1 CGU

4 participants